Skip to content

Too many tests generated by Fuzzer #1225 #1239

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 27, 2022

Conversation

Markoutte
Copy link
Collaborator

Description

The problem was the instrumentation doesn't change bytecode for classes which have space-character in the path on Windows. java.net.URL can keep both options: with character "%20" or " " in the path. Because of this the code source path (that contains %20) cannot be found in the list of sources to be transformed.

Therefore instrumentation doesn't supply any coverage to minimization and plenty of tests are generated.

To fix this, URL is converted into URI first and then full path of file is requested.

Fixes #1225

Type of Change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Automated Testing

All tests should pass

Manual Scenario

Try to reproduce example from the issue. Nearly 29 tests should be generated.

Checklist:

  • The change followed the style guidelines of the UTBot project
  • Self-review of the code is passed
  • The change contains enough commentaries, particularly in hard-to-understand areas
  • New documentation is provided or existed one is altered
  • No new warnings
  • New tests have been added
  • All tests pass locally with my changes

@Markoutte Markoutte merged commit ea14197 into main Oct 27, 2022
@Markoutte Markoutte deleted the pelevin/1225_Too_many_tests_generated_by_Fuzzer branch October 27, 2022 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Too many tests generated by Fuzzer
2 participants